Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync main with deepmind #146

Merged
merged 8 commits into from
Aug 6, 2023
Merged

Sync main with deepmind #146

merged 8 commits into from
Aug 6, 2023

Conversation

thowell
Copy link
Collaborator

@thowell thowell commented Aug 5, 2023

No description provided.

copybara-github and others added 6 commits July 25, 2023 09:01
PiperOrigin-RevId: 550901847
Change-Id: I6ee5c15f2836545f817d6e782331648ac1d5b1db
PiperOrigin-RevId: 551561019
Change-Id: I27e2ae164898264f29429ed007dab0e32d771546
All new tasks should derived from ThreadSafeTask, which will soon be cleaned and renamed.

PiperOrigin-RevId: 551564738
Change-Id: I5a398ef489960aac41afacda861916899ce2ef62
…ad of rolling out.

For the sampling and gradient planners, this will make no difference. For iLQG, this will ignore the feedback terms in the policy.

Also, remove some repetitive code from agent_service_test.cc

PiperOrigin-RevId: 552483307
Change-Id: Ib3f9420b3541a5f77a6abb7c577737169f3780d5
The same functionality is available through the mutex_ protected field, and most subclasses of the Task class shouldn't care about that mutex.

Remove unused includes.

PiperOrigin-RevId: 552719147
Change-Id: Iab85782f9e21643ce47eece26ef1784f8904ae00
@thowell thowell requested a review from erez-tom August 5, 2023 20:17
grpc/agent_service_test.cc Outdated Show resolved Hide resolved
grpc/grpc_agent_util.cc Outdated Show resolved Hide resolved
grpc/grpc_agent_util.cc Outdated Show resolved Hide resolved
grpc/grpc_agent_util.cc Outdated Show resolved Hide resolved
@thowell
Copy link
Collaborator Author

thowell commented Aug 6, 2023

Fixed one small spelling error in the README.

@erez-tom erez-tom merged commit 760202e into google-deepmind:main Aug 6, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants